Skip to content

K8SPG-1080 v3 module#1686

Draft
oksana-grishchenko wants to merge 36 commits into
mainfrom
K8SPG-1080-v3-module
Draft

K8SPG-1080 v3 module#1686
oksana-grishchenko wants to merge 36 commits into
mainfrom
K8SPG-1080-v3-module

Conversation

@oksana-grishchenko

@oksana-grishchenko oksana-grishchenko commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

v3 module

Problem:
K8SPG-1080
PG 3.0.0 is released while go module still references v2 which makes it harder to use it.

Solution:
Implement v3 module

CHECKLIST

Jira

  • Is the Jira ticket created and referenced properly?
  • Does the Jira ticket have the proper statuses for documentation (Needs Doc) and QA (Needs QA)?
  • Does the Jira ticket link to the proper milestone (Fix Version field)?

Tests

  • Is an E2E test/test case added for the new feature/change?
  • Are unit tests added where appropriate?

Config/Logging/Testability

  • Are all needed new/changed options added to default YAML files?
  • Are all needed new/changed options added to the Helm Chart?
  • Did we add proper logging messages for operator actions?
  • Did we ensure compatibility with the previous version or cluster upgrade process?
  • Does the change support oldest and newest supported PG version?
  • Does the change support oldest and newest supported Kubernetes version?

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

Comment thread cmd/postgres-operator/main.go
Comment thread internal/bridge/crunchybridgecluster/postgres_test.go
Comment thread internal/controller/postgrescluster/helpers_test.go
Comment thread internal/controller/postgrescluster/pgmonitor_test.go
Comment thread internal/controller/postgrescluster/snapshots.go
Comment thread internal/controller/standalone_pgadmin/configmap.go
Comment thread internal/controller/standalone_pgadmin/statefulset.go
Comment thread internal/controller/standalone_pgadmin/volume.go
Comment thread percona/controller/pgcluster/owner_ref_migration.go
Comment thread percona/controller/pgrestore/snapshot/reconcile_test.go
Comment thread internal/controller/standalone_pgadmin/configmap.go
Comment thread internal/controller/standalone_pgadmin/service.go
Comment thread hack/create-todo-patch.sh
@@ -9,10 +9,10 @@ import (

var (
// GroupVersion is group version used to register these objects
GroupVersion = schema.GroupVersion{Group: "pgv2.percona.com", Version: "v2"}
GroupVersion = schema.GroupVersion{Group: "pgv3.percona.com", Version: "v2"}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we don't want to change this and don't need to to use v3 in go

Comment thread hack/create-todo-patch.sh
Comment thread hack/create-todo-patch.sh

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 267 out of 267 changed files in this pull request and generated 6 comments.

Comments suppressed due to low confidence (1)

internal/controller/postgrescluster/pki.go:77

  • reconcileRootCertificate no longer checks for the legacy root CA secret name (naming.RootCertSecret), even though that constant is still documented as required for upgrades (K8SPG-555). This can cause an operator upgrade to silently mint a new CA instead of reusing/migrating the existing one, breaking client connections.

Comment thread pkg/apis/upstream.pgv2.percona.com/v1beta1/shared_types.go
Comment thread pkg/apis/upstream.pgv2.percona.com/v1beta1/shared_types.go
Comment thread pkg/apis/upstream.pgv2.percona.com/v1beta1/zz_generated.deepcopy.go
Comment thread pkg/apis/upstream.pgv2.percona.com/v1beta1/zz_generated.deepcopy.go
Comment on lines 50 to 52
FROM registry.access.redhat.com/${BASE_IMAGE}/ubi-minimal AS ubi
RUN microdnf update -y && microdnf clean all -y
RUN microdnf update -y && microdnf clean all -y || true

Comment thread percona/naming/prefix.go
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants